Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Grid syncing, Instat object & File Menu #452

Merged
merged 5 commits into from
Jan 11, 2016

Conversation

dannyparsons
Copy link
Contributor

Implemented new syncing method between grids and Instat object and updated some functions on the file menu (some not working and some incomplete)

dannyparsons added a commit that referenced this pull request Jan 11, 2016
Grid syncing, Instat object & File Menu
@dannyparsons dannyparsons merged commit 3de876e into IDEMSInternational:master Jan 11, 2016
@dannyparsons dannyparsons deleted the grid_syncing branch January 11, 2016 21:34
@dannyparsons dannyparsons restored the grid_syncing branch January 12, 2016 09:08
@deaspo
Copy link
Contributor

deaspo commented Jan 12, 2016

Hi Danny, line 43 of clsGridLink.vb is causing the debugging of the project to crash. Something to do with NullReferenceException. Quick discussion on this needed since I cannot run the project.

@dannyparsons
Copy link
Contributor Author

Hi Polycarp,

Sorry only just saw this. It's not crashing for me so I'll have to see it
with you to see what's going on to resolve it. Let's discuss tomorrow, what
were you doing when it crash?

As a temporary fix, you can comment out lines 43-45 and instead just set
those 3 boolean values to True and that should at least get it running
until we see the problem.

Danny

On 12 January 2016 at 17:50, Polycarp Okock [email protected]
wrote:

Hi Danny, line 43 of clsGridLink.vb is causing the debugging of the
project to crash. Something to do with NullReferenceException. Quick
discussion on this needed since I cannot run the project.


Reply to this email directly or view it on GitHub
#452 (comment)
.

@deaspo
Copy link
Contributor

deaspo commented Jan 13, 2016

I think its probably because the InstantObject was updated, right? The methods get_data_changed() e.t.c I do not have them.

@dannyparsons
Copy link
Contributor Author

The latest version of the Instat Object is in the static folder which you
have when you pull the repo so it should be using that version and find
those methods.

On 13 January 2016 at 05:09, Polycarp Okock [email protected]
wrote:

I think its probably because the InstantObject was updated, right? The
methods get_data_changed() e.t.c I do not have them.


Reply to this email directly or view it on GitHub
#452 (comment)
.

@dannyparsons
Copy link
Contributor Author

Have you determined why the error is appearing?

I think adding:
If frmMain.clsRLink.bInstatObjectExists Then

    End If

before line 43 and until the end of this class is needed because it
shouldn't try to update anything without an Instat object. Whether that
fixes your error or not is a different question..

On 13 January 2016 at 08:29, Danny Parsons [email protected] wrote:

The latest version of the Instat Object is in the static folder which you
have when you pull the repo so it should be using that version and find
those methods.

On 13 January 2016 at 05:09, Polycarp Okock [email protected]
wrote:

I think its probably because the InstantObject was updated, right? The
methods get_data_changed() e.t.c I do not have them.


Reply to this email directly or view it on GitHub
#452 (comment)
.

@dannyparsons dannyparsons deleted the grid_syncing branch January 14, 2016 10:17
dannyparsons added a commit that referenced this pull request Jan 7, 2017
dannyparsons pushed a commit that referenced this pull request May 27, 2017
maxwellfundi added a commit to Wycklife/R-Instat that referenced this pull request Jun 26, 2019
Reusing the tryScript() function code from model dialogue in Hypothes…
lloyddewit pushed a commit that referenced this pull request Aug 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants